Skip to content

feat(reference-app): re-source the README walkthrough from the /demo flow - #195

Merged
ardetrick merged 1 commit into
mainfrom
feat/reference-app-walkthrough-resource
Jul 26, 2026
Merged

feat(reference-app): re-source the README walkthrough from the /demo flow#195
ardetrick merged 1 commit into
mainfrom
feat/reference-app-walkthrough-resource

Conversation

@ardetrick

Copy link
Copy Markdown
Owner

Replaces the functional tests' echo-endpoint captures (a random authorization code, unstyled) with the styled /demo flow: login → consent → callback → the real token response. The quickStart test now screenshots the consent step and skips the client-listing landing page.

  • Drops the remember-me section — it's clutter for a landing README; revisit with a richer gif/slideshow later (its test still runs).
  • Renames the flow directory to authorization-code-flow, and the section becomes a single "See it work" walkthrough that ends on the actual token response instead of "the client will have a JWT (not pictured)."
  • Every committed screenshot is now deterministic (verified byte-identical run to run) and a uniform 576x520.

Follow-up (not here): the functional test's build screenshot directory accumulates stale files when step names change, so regenerating after a rename needs a clean; the task or test should clear its output dir first.

…flow

Replaces the functional tests' echo-endpoint captures with the styled /demo flow: login, consent, callback, and the real token response. The quickStart test now screenshots the consent step and skips the client-listing landing page. Drops the remember-me section (revisit with a richer gif/slideshow later) and renames the flow directory to authorization-code-flow. Every screenshot is now deterministic and fixed-size, and the walkthrough ends on the token payoff instead of "not pictured".
@ardetrick
ardetrick merged commit 67a2dca into main Jul 26, 2026
1 check passed
@ardetrick
ardetrick deleted the feat/reference-app-walkthrough-resource branch July 26, 2026 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant